Conversation
bukkit/src/main/java/com/github/games647/fastlogin/bukkit/BungeeManager.java
Outdated
Show resolved
Hide resolved
6d3932f to
61b9cb1
Compare
|
Yeah currently the |
|
Thanks for separating the commits. I cherry-picked your first commit until we look into the other manner. Thanks for the contribution. |
|
Do you plan to do the refactoring job or do you want me to do something here? One difference between Bungeecord and Velocity implementation is that the |
Originally the plan was that I'll do it in combination with my in-progress session manager changes. However, I didn't have the time yet. Maybe smaller refactoring steps should be better in this case. So refactoring only the floodgate+velocity in this PR would be enough. If you could do it, that would be great. |
|
Any update on this? |
Based on: TuxCoding#649
Sorry, no time for that. But I see that you have started working on it already, so good luck 👍 |
Based on: TuxCoding#649 Solve hooks not found Add defination of floodgateservice Geyser API
Based on: TuxCoding#649 Solve hooks not found Add defination of floodgateservice Geyser API
Based on: TuxCoding#649 Solve hooks not found Add defination of floodgateservice Geyser API
|
@xrh0905 or @Smart123s don't you have working release that fixes this issue, please? |
|
@bobhenl Yeah, I did that, and it's got merged already. |
|
@xrh0905 so it's in the released official Dev build already? I've tried it on velocity and it wasn't working, even when I tried to build these two mentioned pull requests from you and @Smart123s , your code worked only for registration but not for auto login for geyser floodgate users. May I done it wrong. Can you provide the working build jar if you have please? |
|
@bobhenl Well actually it's been a while since last player join my server with bedrock edition so maybe something is wrong with my code. |
|
@xrh0905 yeah it looks there's definitely something wrong, you can even test it & you'll see that it's not working with velocity. It auto register them, but not auto login. |
|
Pleasseeeeeeeeeeeeeeeeeeeeeee implement thisssssssssssssss |
Summary of your change
Verified with:
Note: When using Velocity+FloodGate+FastLogin combo behind Geyser, this needs another fix on the FloodGate plugin side. Modern IP forwarding is broken and FloodGate player name is evaluated without a configured Bedrock-player prefix.
Related issue
Related GeyserMC/Floodgate#227